GDK-Win32: Remove extraneous call to _gdk_win32_append_event()
authorChun-wei Fan <fanchunwei@src.gnome.org>
Wed, 17 Mar 2021 03:09:37 +0000 (11:09 +0800)
committerChun-wei Fan <fanchunwei@src.gnome.org>
Wed, 17 Mar 2021 03:22:23 +0000 (11:22 +0800)
Apparently, by comparing with the other backends, we should not call
_gdk_win32_append_event() after calling gdk_scroll_event_new() but we should
call it after calling gdk_scroll_event_new_discrete(), which was why we didn't
restore the cursor after we scroll using the mouse wheel and didn't manage to
remove the shade that appears after we scrolled to the very top or very bottom.

Also, as suggested by the reporter, use IDC_SIZEALL for the system cursor that
we fall back to if no cursor theme is installed, as with other Windows
programs.

This should really fix issue #3581.

gdk/win32/gdkcursor-win32.c
gdk/win32/gdkevents-win32.c

index fe9f299c719b3eb93113eb2790e86266812b5540..61cf9d83a1b65f929a433970572b4fcd6b2eadb6 100644 (file)
@@ -61,7 +61,7 @@ static DefaultCursor default_cursors[] = {
   { "text", IDC_IBEAM },
   { "move", IDC_SIZEALL },
   { "not-allowed", IDC_NO },
-  { "all-scroll", IDC_ARROW },
+  { "all-scroll", IDC_SIZEALL },
   { "ew-resize", IDC_SIZEWE },
   { "e-resize", IDC_SIZEWE },
   { "w-resize", IDC_SIZEWE },
index 9c47263f07f686660630b21428159b1e9e74b2c3..5a17522ae196ab076a92c5213e0b36c7d10f76d9 100644 (file)
@@ -2485,8 +2485,6 @@ gdk_event_translate (MSG *msg,
                                     delta_y,
                                     FALSE);
 
-      _gdk_win32_append_event (event);
-
       /* Append the discrete version too */
       direction = 0;
       if (msg->message == WM_MOUSEWHEEL)